Skip to content

Use redis for the discord member cache over mongo#297

Merged
ChrisLovering merged 5 commits into
mainfrom
redis-member-cache
Jul 19, 2024
Merged

Use redis for the discord member cache over mongo#297
ChrisLovering merged 5 commits into
mainfrom
redis-member-cache

Conversation

@ChrisLovering
Copy link
Copy Markdown
Member

@ChrisLovering ChrisLovering commented Jul 16, 2024

The secret for this is already deployed to the cluster

@ChrisLovering ChrisLovering requested a review from jb3 July 16, 2024 18:08
shtlrs
shtlrs previously approved these changes Jul 16, 2024
Copy link
Copy Markdown

@shtlrs shtlrs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have not tested this, but it looks safe and good to me.

@jchristgit A dream come true, mongo is going bongo

Comment thread backend/discord.py Outdated
Comment thread backend/constants.py
Copy link
Copy Markdown

@vivekashok1221 vivekashok1221 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ChrisLovering ChrisLovering merged commit 5023467 into main Jul 19, 2024
@ChrisLovering ChrisLovering deleted the redis-member-cache branch July 19, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants